I think this is a win cc/ @josharian @AndreasArvidsson

@wenkokke this will make some of your test files shorter

i'm a tentative +1. maybe a +0.

i'm a tentative +1. maybe a +0.

Looking at the fixtures themselves, rather than the diffs, I feel the same. I do feel it makes some tests a lot easier to read, but can see how it might slightly reduce clarity on others

I think this is an improvement. I do wonder if we also should remove empty range lines? Having those empty lines in the middle of the code block doesn't really help you with readability. Also now that we exclude lines outside of the range I'm tempted to go back to having the start range be above the code.

source:

class MyClass {
  myFunk() {

  }
}

scope:

    >----------
1|   myFunk() {
2|
3|   }
   ---<

I disagree on both counts 😅. Shall we aim to discuss? I'd be inclined to merge this in the meantime as I think we all tentatively agree it's an improvement?

Ok with me

Sounds good. I'll merge once you give it the stamp